Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: Update German translations for 0.6.4 #5722

Merged
merged 3 commits into from
Jul 19, 2024

Conversation

suntorytimed
Copy link
Contributor

Describe the changes you have conducted here

Preview the messages on https://inlang.com/github.com/suntorytimed/AppFlowy .

@suntorytimed
Copy link
Contributor Author

suntorytimed commented Jul 11, 2024

@LucasXu0 please note that I translated web.signInAgreement including web.termOfUse, web.and and web.privacyPolicy inside of the translated text, as it isn't possible to translate the Text to German with the sentence ending on "[...]AppFlowys [Terms] [and] [Privacy Policy]". So the three other strings will be duplicated and the translation in Germany will look weird, as long as the strings are handled like this in AppFlowy.

And something should happen with this itemThree in the English translation. InLang automatically optimizes the translation and always removes it.

Copy link

codecov bot commented Jul 11, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.27%. Comparing base (c122f0a) to head (3936e8c).
Report is 62 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5722      +/-   ##
==========================================
- Coverage   86.74%   83.27%   -3.48%     
==========================================
  Files         217      214       -3     
  Lines        5032     5290     +258     
  Branches      831      863      +32     
==========================================
+ Hits         4365     4405      +40     
- Misses        561      795     +234     
+ Partials      106       90      -16     
Flag Coverage Δ
appflowy_web_app 83.27% <ø> (-3.48%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@LucasXu0
Copy link
Collaborator

And something should happen with this itemThree in the English translation. InLang automatically optimizes the translation and always removes it.

Because itemThree is empty, I should remove it. cc @Xazin

please note that I translated web.signInAgreement including web.termOfUse, web.and and web.privacyPolicy inside of the translated text

I want to confirm if you can translate it to German correctly with this format:

"signInAgreement": "By clicking "Continue" above, you confirm that
you have read, understood, and agreed to
AppFlowy's {} and {}",
"privacyPolicy": "Privacy Policy",
"termOfUse": "Terms",

the {} means the linked text and I'll fill it up with "privacyPolicy" and "termOfUse".

@suntorytimed
Copy link
Contributor Author

I want to confirm if you can translate it to German correctly with this format:

"signInAgreement": "By clicking "Continue" above, you confirm that
you have read, understood, and agreed to
AppFlowy's {} and {}",
"privacyPolicy": "Privacy Policy",
"termOfUse": "Terms",

the {} means the linked text and I'll fill it up with "privacyPolicy" and "termOfUse".

Yes, I would be able to translate it properly in this format.

@suntorytimed suntorytimed changed the title Update German translations for 0.6.4 chore: Update German translations for 0.6.4 Jul 12, 2024
@LucasXu0
Copy link
Collaborator

@suntorytimed The way I mentioned doesn't work. If using the translation like that, it's unable to set the terms and privacy link.

          TextSpan(
            text: LocaleKeys.web_signInAgreement.tr(
              args: [
                LocaleKeys.web_termOfUse.tr(),
                LocaleKeys.web_privacyPolicy.tr(),
              ],
            ),
            style: const TextStyle(color: Colors.grey, fontSize: 12),
          ),

@suntorytimed
Copy link
Contributor Author

@LucasXu0 that's unfortunate. I will try to formulate the text different in the German translation without moving too far away from the original meaning. Will sound weird for a native speaker, but at least it won't have trailing strings.

@suntorytimed suntorytimed changed the title chore: Update German translations for 0.6.4 DRAFT chore: Update German translations for 0.6.4 Jul 12, 2024
@suntorytimed
Copy link
Contributor Author

Not ideal, but this should work. It basically refers to a list of "documents" that need to be read, understood and accepted and "AppFlowy's Privacy Policy and Terms" would be the one and only entry of the "list"

@suntorytimed
Copy link
Contributor Author

Does Commit messages lint / commitlint (pull_request) complain about Fix wrongfully removed itemThree not having a tag like chore:?

@suntorytimed suntorytimed changed the title DRAFT chore: Update German translations for 0.6.4 chore: Update German translations for 0.6.4 Jul 12, 2024
@Xazin
Copy link
Collaborator

Xazin commented Jul 12, 2024

And something should happen with this itemThree in the English translation. InLang automatically optimizes the translation and always removes it.

Because itemThree is empty, I should remove it. cc @Xazin

It's removed in our plan+billing changes, but the PR won't be opened before Wednesday morning earliest.

@suntorytimed
Copy link
Contributor Author

rebased and hopefully fixed the commit message for the CI

@LucasXu0 LucasXu0 added translation tasks related to translating AppFlowy into different languages v0.6.5 labels Jul 19, 2024
@LucasXu0 LucasXu0 merged commit 9361afd into AppFlowy-IO:main Jul 19, 2024
17 of 18 checks passed
zoli added a commit to zoli/AppFlowy that referenced this pull request Jul 22, 2024
* main:
  fix: integration test failed (grid row detail page: hide and show hidden fields) (AppFlowy-IO#5781)
  feat: ai billing (AppFlowy-IO#5741)
  fix: can not display rows when rows overthan 1000 (AppFlowy-IO#5777)
  feat: support publish database on web (AppFlowy-IO#5748)
  feat: publish databse to Web (AppFlowy-IO#5709)
  chore: update Spanish translations (AppFlowy-IO#5742)
  chore: update Chinese translations (AppFlowy-IO#5727)
  fix: Add retry for admin client sign in for test (AppFlowy-IO#5767)
  chore: update Hebrew translation (AppFlowy-IO#5738)
  chore: update German translations (AppFlowy-IO#5722)
  chore: update Russian translations (AppFlowy-IO#5730)
  chore: update build config (AppFlowy-IO#5759)
  chore: enable local ai and local ai chat (AppFlowy-IO#5755)
zoli added a commit to zoli/AppFlowy that referenced this pull request Jul 22, 2024
* main:
  fix: integration test failed (grid row detail page: hide and show hidden fields) (AppFlowy-IO#5781)
  feat: ai billing (AppFlowy-IO#5741)
  fix: can not display rows when rows overthan 1000 (AppFlowy-IO#5777)
  feat: support publish database on web (AppFlowy-IO#5748)
  feat: publish databse to Web (AppFlowy-IO#5709)
  chore: update Spanish translations (AppFlowy-IO#5742)
  chore: update Chinese translations (AppFlowy-IO#5727)
  fix: Add retry for admin client sign in for test (AppFlowy-IO#5767)
  chore: update Hebrew translation (AppFlowy-IO#5738)
  chore: update German translations (AppFlowy-IO#5722)
  chore: update Russian translations (AppFlowy-IO#5730)
  chore: update build config (AppFlowy-IO#5759)
  chore: enable local ai and local ai chat (AppFlowy-IO#5755)
  chore: bump version 0.6.4 (AppFlowy-IO#5744)
  fix: improve color selector (AppFlowy-IO#5743)
  fix: reset space relationship when clearing cache (AppFlowy-IO#5737)
  chore: show plugin state (AppFlowy-IO#5740)
  chore: download llm files (AppFlowy-IO#5723)
  feat: optimize the read recent views speed (AppFlowy-IO#5726)
  chore: fix compile (AppFlowy-IO#5733)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
translation tasks related to translating AppFlowy into different languages v0.6.5
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants